Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix NoSuchMethodError when projects have OkHttp 3 dependency #342

Merged
merged 2 commits into from
Apr 5, 2021

Conversation

jimmyjames
Copy link
Contributor

Changes

After updating to OkHttp v4, customers may experience NoSuchMethodError when making requests with this library, if OkHttp v3 is taking precedence on the runtime classpath. Given that OkHttp v4 is binary compatible with OkHttp v3, this change reverts to using (now-deprecated) OkHttp methods that do not exist in v4.

We cannot guarantee that we will never use any OkHttp v4 APIs, and the resolution to such issues may require managing the dependencies as discussed in #324. But for now, we can alleviate this issue for the majority of customers with this change.

References

@jimmyjames jimmyjames added this to the v1-Next milestone Apr 5, 2021
@jimmyjames jimmyjames requested a review from a team as a code owner April 5, 2021 16:10
@jimmyjames jimmyjames added the review:small Small review label Apr 5, 2021
@jimmyjames jimmyjames merged commit 3298cfd into master Apr 5, 2021
@jimmyjames jimmyjames modified the milestones: v1-Next, 1.28.1 Apr 5, 2021
@jimmyjames jimmyjames deleted the use-deprecated-okhttp-methods branch August 10, 2021 22:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants